Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

make --secondary-server and --ssl optional #18

Merged
merged 1 commit into from
Mar 9, 2020
Merged

make --secondary-server and --ssl optional #18

merged 1 commit into from
Mar 9, 2020

Conversation

ctaggart
Copy link
Contributor

@ctaggart ctaggart commented Mar 9, 2020

az vmware privatecloud addidentitysource -- few parameters such as “--ssl or --secondary-server” are not required params in Azure portal unlike in CLI. These should be optional in CLI as well, isn’t it?

I made them optional to match the UI. ssl is not optional in the API, but it now default to Disabled like the UI.

@ctaggart ctaggart added this to the 0.5.0 milestone Mar 9, 2020
@ctaggart ctaggart merged commit 87ee0f9 into master Mar 9, 2020
@ctaggart ctaggart deleted the optional branch March 9, 2020 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant